Skip to content

Clarify rules for rel="noopener" and cross origin #39675

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Khaspper
Copy link
Contributor

Description

I clarified that rel="noopener" should be used for cross-origin links

Motivation

This is my first time contributing to an OSP!!! And MDN Docs has helped me a lot with my Web Dev journey and I would like to thank them by contributing to their OSP!!

Additional details

I saw an issue targetting this issue so I decided it would be perfect for my first contribution so I'm making these changes because it clarifies when and when not to use noopener in your HTML file!

Related issues and pull requests

Fixes #39344

@Khaspper Khaspper requested a review from a team as a code owner May 25, 2025 23:29
@Khaspper Khaspper requested review from pepelsbey and removed request for a team May 25, 2025 23:29
@github-actions github-actions bot added Content:Other Any docs not covered by another "Content:" label size/xs [PR only] 0-5 LoC changed labels May 25, 2025
Copy link
Contributor

@CharlesBelov
Copy link

Thank you for this. I'm not knowledgeable about the subject to be able to respond authoritatively, but your content is understandable and is consistent with my experience with text fragment links missing the rel="noopener" attribute.

Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, thank you; and congrats on your first contribution here!

@Josh-Cena Josh-Cena merged commit 8d160cd into mdn:main May 31, 2025
7 checks passed
@Khaspper
Copy link
Contributor Author

Thank you! :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Other Any docs not covered by another "Content:" label size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Is wording regarding rel="noopener" for text fragment links complete?
3 participants